Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add logic to generate item links and getitemlink script command #3236

Merged
merged 5 commits into from
Oct 16, 2023

Conversation

guilherme-gm
Copy link
Member

@guilherme-gm guilherme-gm commented Oct 6, 2023

Pull Request Prelude

Changes Proposed

Implements the logic to generate "item links" on the server side (like when a player shift-clicks an item in-game). This uses a client-specific base62 string representing the item, refine, cards, etc.

With this PR it will be possible to create item links using script and source (which will be needed for some features)

Special thanks to Asheraf for sharing references with me for the code used in clif; Also credits to rA since I looked into some of their implementation to do this.

Issues addressed: None, I think

@guilherme-gm guilherme-gm changed the title logic to generate item links and getitemlink script command Add logic to generate item links and getitemlink script command Oct 6, 2023
src/map/clif.c Outdated Show resolved Hide resolved
@MishimaHaruna MishimaHaruna added this to the Release v2023.10 milestone Oct 14, 2023
src/map/clif.c Outdated Show resolved Hide resolved
this function makes it simpler to get a value from a number script
array, saving users from needing to handle the stack directly
this command creates a client-specific formatted tag with item info
@MishimaHaruna MishimaHaruna merged commit 3ac3c97 into HerculesWS:master Oct 16, 2023
4 of 254 checks passed
@guilherme-gm guilherme-gm deleted the itemlink branch March 2, 2024 21:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants